Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Description and License sections to Readme + Documented backend functions #72

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ercz16
Copy link
Collaborator

@ercz16 ercz16 commented Sep 29, 2023

Issue

Missing clear goals and descriptions of the app. Functions could include more comments to make revisions easier for first timers.

Description

Added a description to readme.md that explicitly states what the app wants to achieve. Includes goals, motivations, and ways to expand app functionality. Added a possible Apache license that can be used to clearly state the fair usage of the software. Added comments for backend functions that are confusing at first, especially for developers unfamiliar with Golang.

Type Of Change

  • This change is purely a documentation update

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

Copy link
Contributor

@kthchew kthchew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some suggestions/questions:

@@ -65,4 +92,205 @@
- [Expo Documentation](https://docs.expo.dev/)
- [Golang Documentation](https://go.dev/doc/)


## License
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

probably best to put in a separate LICENSE or COPYING file instead of the README.md (if you want a mention of Apache in the README that's probably fine), I'm not sure if GitHub will auto detect the license if it's not (and it's a lot of stuff in the README that most people won't read)

Also, just curious, why Apache? Most of the other OSC projects seem to be MIT, but some are AGPL or GPL. I don't have a strong preference either way but was just wondering

Comment on lines +5 to +12
1[Description](#description)
2[Install](#install)
3[Usage](#usage)
4[Debugging](#debugging)
5[Contributing](#contributing)
- [Front End](#front-end)
- [Tips](#tips)
6[License](#license)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this change to the list intentional? Doesn't look as nice of a list to me imo

result, err := database.Execute("INSERT INTO Accounts (email, password) VALUES (?, SHA2(?, 256))", acc.Email, acc.Password)
if err != nil {
return
}

// Check if any rows were affected by the database operation.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you want to comment this it's probably more effective to say that no rows affected = account already exists (iirc)

Comment on lines +20 to +36
Key Features:
Discover Lunch Partners: Browse through profiles of fellow students from various fields of study and interests.
Schedule Lunches: Arrange meetups at your convenience, allowing you to make the most of your break time.
Expand Your Network: Connect with individuals in your field of study, fostering opportunities for future collaborations and mentorships.
Enhanced Social Experience: Share your interests, hobbies, and career aspirations to find like-minded individuals.
Safe and Secure: Your privacy and safety are our top priorities. Enjoy a secure environment for making connections.
How It Works:
Create Your Profile: Sign up and build a personal profile highlighting your interests, major, and availability.
Find Lunch Partners: Browse profiles and connect with students who share your interests or career goals.
Schedule a Lunch: Propose meetups and set the date, time, and location for your lunch get-togethers.
Meet and Connect: Enjoy meaningful conversations and make lasting connections with your lunch buddies.
Why Lunch Buddies?
Networking Opportunities: Expand your network and gain insights into your chosen field of study or career path.
Friendship Building: Forge friendships that extend beyond your lunch meetups.
Campus Community: Strengthen the sense of community among your fellow students.
Personal Growth: Step out of your comfort zone and enhance your social skills.
Join Lunch Buddies today and start building connections, one meal at a time. Whether you're an undergraduate, graduate student, or simply hungry for friendship, we're here to help you find your lunchtime companions.
Copy link
Contributor

@kthchew kthchew Nov 8, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Key Features:
Discover Lunch Partners: Browse through profiles of fellow students from various fields of study and interests.
Schedule Lunches: Arrange meetups at your convenience, allowing you to make the most of your break time.
Expand Your Network: Connect with individuals in your field of study, fostering opportunities for future collaborations and mentorships.
Enhanced Social Experience: Share your interests, hobbies, and career aspirations to find like-minded individuals.
Safe and Secure: Your privacy and safety are our top priorities. Enjoy a secure environment for making connections.
How It Works:
Create Your Profile: Sign up and build a personal profile highlighting your interests, major, and availability.
Find Lunch Partners: Browse profiles and connect with students who share your interests or career goals.
Schedule a Lunch: Propose meetups and set the date, time, and location for your lunch get-togethers.
Meet and Connect: Enjoy meaningful conversations and make lasting connections with your lunch buddies.
Why Lunch Buddies?
Networking Opportunities: Expand your network and gain insights into your chosen field of study or career path.
Friendship Building: Forge friendships that extend beyond your lunch meetups.
Campus Community: Strengthen the sense of community among your fellow students.
Personal Growth: Step out of your comfort zone and enhance your social skills.
Join Lunch Buddies today and start building connections, one meal at a time. Whether you're an undergraduate, graduate student, or simply hungry for friendship, we're here to help you find your lunchtime companions.
### Key Features
- Discover Lunch Partners: Browse through profiles of fellow students from various fields of study and interests.
- Schedule Lunches: Arrange meetups at your convenience, allowing you to make the most of your break time.
- Expand Your Network: Connect with individuals in your field of study, fostering opportunities for future collaborations and mentorships.
- Enhanced Social Experience: Share your interests, hobbies, and career aspirations to find like-minded individuals.
- Safe and Secure: Your privacy and safety are our top priorities. Enjoy a secure environment for making connections.
### How It Works
1. Create Your Profile: Sign up and build a personal profile highlighting your interests, major, and availability.
2. Find Lunch Partners: Browse profiles and connect with students who share your interests or career goals.
3. Schedule a Lunch: Propose meetups and set the date, time, and location for your lunch get-togethers.
4. Meet and Connect: Enjoy meaningful conversations and make lasting connections with your lunch buddies.
### Why Lunch Buddies?
- Networking Opportunities: Expand your network and gain insights into your chosen field of study or career path.
- Friendship Building: Forge friendships that extend beyond your lunch meetups.
- Campus Community: Strengthen the sense of community among your fellow students.
- Personal Growth: Step out of your comfort zone and enhance your social skills.
Join Lunch Buddies today and start building connections, one meal at a time. Whether you're an undergraduate, graduate student, or simply hungry for friendship, we're here to help you find your lunchtime companions.

An empty line is needed between each line, otherwise GitHub's Markdown viewer will show this entire block as one big paragraph with no line breaks

Or a bulleted list could be used with no empty lines

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants